Skip to content

feat: move configuration out of mongodb - #846

Merged
Swofty-Developments merged 5 commits into
Swofty-Developments:masterfrom
ArikSquad:agent/fairysouls-configuration
Aug 9, 2026
Merged

feat: move configuration out of mongodb #846
Swofty-Developments merged 5 commits into
Swofty-Developments:masterfrom
ArikSquad:agent/fairysouls-configuration

Conversation

@ArikSquad

@ArikSquad ArikSquad commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Replace the CSVs with a version-controlled YAML catalog
  • Include the current island locations and server types, including newer islands missing from the old configuration.
  • Remove static fairy-soul location data from MongoDB and load it from application resources instead.
  • Update related handlers, GUI views, documentation, and tests.

Why

Static location data is configuration and should be deployed/versioned with the server. MongoDB remains available for player progress persistence.

Validation

  • ./gradlew :type.skyblockgeneric:test --tests net.swofty.type.skyblockgeneric.data.fairysouls.FairySoulCatalogTest
  • git diff --check

@vercel

vercel Bot commented Aug 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
hypixel-skyblock Ignored Ignored Aug 8, 2026 4:19pm

Comment thread website/docs/setup/game-servers.md Outdated
Comment thread website/docs/troubleshooting.md
@ArikSquad

Copy link
Copy Markdown
Collaborator Author

also regions should probably be like this too

@ArikSquad
ArikSquad marked this pull request as ready for review August 7, 2026 17:53
@ArikSquad ArikSquad changed the title feat: update fairy souls configuration feat: move configuration out of mongodb Aug 7, 2026
@ArikSquad
ArikSquad marked this pull request as draft August 8, 2026 13:30
@ArikSquad
ArikSquad force-pushed the agent/fairysouls-configuration branch from 08abb69 to 25dcbc9 Compare August 8, 2026 16:17
@ArikSquad
ArikSquad marked this pull request as ready for review August 8, 2026 16:17
@Swofty-Developments
Swofty-Developments merged commit 3789d0c into Swofty-Developments:master Aug 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants